Skip to content

Differentiate unset value from null value in OpenApiSchema.Const#2936

Merged
baywet merged 8 commits into
microsoft:mainfrom
Youssef1313:dev/ygerges/const
Jul 14, 2026
Merged

Differentiate unset value from null value in OpenApiSchema.Const#2936
baywet merged 8 commits into
microsoft:mainfrom
Youssef1313:dev/ygerges/const

Conversation

@Youssef1313

Copy link
Copy Markdown
Member

Fixes #2935

@Youssef1313
Youssef1313 marked this pull request as ready for review July 14, 2026 07:40
@Youssef1313
Youssef1313 requested a review from a team as a code owner July 14, 2026 07:40
@Youssef1313

Copy link
Copy Markdown
Member Author

Benchmark is reporting a perf regression. @baywet I assume it's because we are introducing a new field (bool) to the objects which is increasing the object size? Should it be fine?

baywet
baywet previously approved these changes Jul 14, 2026

@baywet baywet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! Would you mind updating the performance reports please?

@Youssef1313

Copy link
Copy Markdown
Member Author

@copilot Can you update the performance report?

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Youssef1313 and others added 2 commits July 14, 2026 13:53
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
@baywet
baywet merged commit 07b525f into microsoft:main Jul 14, 2026
9 checks passed
@Youssef1313
Youssef1313 deleted the dev/ygerges/const branch July 14, 2026 12:49
baywet added a commit that referenced this pull request Jul 14, 2026
fix: differentiate unset value from null value in OpenApiSchema.Const (#2936)
github-actions Bot pushed a commit to psmfd/agent-expertise-api that referenced this pull request Jul 20, 2026
Updated [Microsoft.OpenApi](https://github.com/Microsoft/OpenAPI.NET)
from 2.10.0 to 2.11.0.

<details>
<summary>Release notes</summary>

_Sourced from [Microsoft.OpenApi's
releases](https://github.com/Microsoft/OpenAPI.NET/releases)._

## 2.11.0

##
[2.11.0](microsoft/OpenAPI.NET@v2.10.0...v2.11.0)
(2026-07-15)


### Features

* adds support for anchor and id external resolution
([557bd6a](microsoft/OpenAPI.NET@557bd6a))
* adds support for anchor and id external resolution
([4138e47](microsoft/OpenAPI.NET@4138e47))
* **schema:** resolve bare $dynamicRef via $dynamicAnchor index
([#​2913](microsoft/OpenAPI.NET#2913))
([baf8428](microsoft/OpenAPI.NET@baf8428))
* **schema:** resolve bare $dynamicRef via $dynamicAnchor index
([#​2913](microsoft/OpenAPI.NET#2913))
([a139f83](microsoft/OpenAPI.NET@a139f83))
* support relative URI resolution in $dynamicRef
([#​2928](microsoft/OpenAPI.NET#2928))
([#​2945](microsoft/OpenAPI.NET#2945))
([6091f5a](microsoft/OpenAPI.NET@6091f5a))
* support relative URI resolution in $dynamicRef
([#​2928](microsoft/OpenAPI.NET#2928))
([#​2945](microsoft/OpenAPI.NET#2945))
([a8688dc](microsoft/OpenAPI.NET@a8688dc))


### Bug Fixes

* adds explicit error message for invalid json pointers
([#​2955](microsoft/OpenAPI.NET#2955))
([a304e56](microsoft/OpenAPI.NET@a304e56))
* differentiate unset value from null value in OpenApiSchema.Const
([#​2936](microsoft/OpenAPI.NET#2936))
([a8787af](microsoft/OpenAPI.NET@a8787af))
* differentiate unset value from null value in OpenApiSchema.Const
([#​2936](microsoft/OpenAPI.NET#2936))
([e08570f](microsoft/OpenAPI.NET@e08570f))
* handle nullability more accurately during serialization for 3.0/2.0
([#​2933](microsoft/OpenAPI.NET#2933))
([bc11356](microsoft/OpenAPI.NET@bc11356))
* handle nullability more accurately during serialization for 3.0/2.0
([#​2933](microsoft/OpenAPI.NET#2933))
([310b6e2](microsoft/OpenAPI.NET@310b6e2))

Commits viewable in [compare
view](microsoft/OpenAPI.NET@v2.10.0...v2.11.0).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Microsoft.OpenApi&package-manager=nuget&previous-version=2.10.0&new-version=2.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This was referenced Jul 20, 2026
github-actions Bot pushed a commit to EelcoLos/nx-tinkering that referenced this pull request Jul 21, 2026
Updated [Microsoft.OpenApi](https://github.com/Microsoft/OpenAPI.NET)
from 2.10.0 to 2.11.0.

<details>
<summary>Release notes</summary>

_Sourced from [Microsoft.OpenApi's
releases](https://github.com/Microsoft/OpenAPI.NET/releases)._

## 2.11.0

##
[2.11.0](microsoft/OpenAPI.NET@v2.10.0...v2.11.0)
(2026-07-15)


### Features

* adds support for anchor and id external resolution
([557bd6a](microsoft/OpenAPI.NET@557bd6a))
* adds support for anchor and id external resolution
([4138e47](microsoft/OpenAPI.NET@4138e47))
* **schema:** resolve bare $dynamicRef via $dynamicAnchor index
([#​2913](microsoft/OpenAPI.NET#2913))
([baf8428](microsoft/OpenAPI.NET@baf8428))
* **schema:** resolve bare $dynamicRef via $dynamicAnchor index
([#​2913](microsoft/OpenAPI.NET#2913))
([a139f83](microsoft/OpenAPI.NET@a139f83))
* support relative URI resolution in $dynamicRef
([#​2928](microsoft/OpenAPI.NET#2928))
([#​2945](microsoft/OpenAPI.NET#2945))
([6091f5a](microsoft/OpenAPI.NET@6091f5a))
* support relative URI resolution in $dynamicRef
([#​2928](microsoft/OpenAPI.NET#2928))
([#​2945](microsoft/OpenAPI.NET#2945))
([a8688dc](microsoft/OpenAPI.NET@a8688dc))


### Bug Fixes

* adds explicit error message for invalid json pointers
([#​2955](microsoft/OpenAPI.NET#2955))
([a304e56](microsoft/OpenAPI.NET@a304e56))
* differentiate unset value from null value in OpenApiSchema.Const
([#​2936](microsoft/OpenAPI.NET#2936))
([a8787af](microsoft/OpenAPI.NET@a8787af))
* differentiate unset value from null value in OpenApiSchema.Const
([#​2936](microsoft/OpenAPI.NET#2936))
([e08570f](microsoft/OpenAPI.NET@e08570f))
* handle nullability more accurately during serialization for 3.0/2.0
([#​2933](microsoft/OpenAPI.NET#2933))
([bc11356](microsoft/OpenAPI.NET@bc11356))
* handle nullability more accurately during serialization for 3.0/2.0
([#​2933](microsoft/OpenAPI.NET#2933))
([310b6e2](microsoft/OpenAPI.NET@310b6e2))

Commits viewable in [compare
view](microsoft/OpenAPI.NET@v2.10.0...v2.11.0).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Microsoft.OpenApi&package-manager=nuget&previous-version=2.10.0&new-version=2.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Eelco Los <5102501+EelcoLos@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenApiSchema.Const cannot differentiate between "null" and unset value.

3 participants